Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

reduce overhead index requests #342

Merged
merged 5 commits into from
Dec 19, 2023
Merged

reduce overhead index requests #342

merged 5 commits into from
Dec 19, 2023

Conversation

shanghaikid
Copy link
Collaborator

@shanghaikid shanghaikid commented Dec 16, 2023

This pr will reduce the problem described in #338 by

  1. build another getCollectionInfo api to get collection info without index info which is slower to call.
  2. rename getCollection api to getCollectionWithIndexInfo.
  3. if no index info is required, replace getCollection with getCollectionInfo in the page.
  4. upgrade package lru-cache to the latest version
  5. use a cache for describeIndex, will delete cache if create/drop the index.
  6. support clearCache, if user clicks the refresh button on collection list page.

Signed-off-by: ruiyi.jiang <ruiyi.jiang@zilliz.com>
Signed-off-by: ruiyi.jiang <ruiyi.jiang@zilliz.com>
shanghaikid and others added 3 commits December 17, 2023 08:32
Signed-off-by: ruiyi.jiang <ruiyi.jiang@zilliz.com>
Signed-off-by: ruiyi.jiang <ruiyi.jiang@zilliz.com>
Signed-off-by: ryjiang <jiangruiyi@gmail.com>
@shanghaikid shanghaikid removed the request for review from nameczz December 19, 2023 08:18
@shanghaikid shanghaikid merged commit ff49222 into main Dec 19, 2023
1 check passed
@shanghaikid shanghaikid deleted the attu-338 branch December 19, 2023 10:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant